Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CGO link error #90

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

CGO link error #90

wants to merge 2 commits into from

Conversation

gguoss
Copy link

@gguoss gguoss commented Feb 26, 2018

build in go version 1.10: pass

@sammy007
Copy link
Owner

sammy007 commented Mar 7, 2018

I will look at it later after fork. There will be an update for just merged CN variants also. For now it's golang 1.6-1.9 only because it's a default version in latest Ubuntu LTS.

@enerc
Copy link

enerc commented Mar 24, 2018

Passed with the above changes and golang 1.10, but I have to pass LD_LIBRARY_PATH for the libs:
LD_LIBRARY_PATH=./cnutil:./hashing build/bin/monero-stratum .....
Solo mining looks working, but still waiting AEON testnet at V7 to test with new crypto algorithm.

@enerc
Copy link

enerc commented Mar 25, 2018

A couple of changes required to make it run with the just released rebased AEON with POW V7:

  • cn_slow_hash has now 6 parameters, instead of 4 (int variant, int prehashed added). Pass variant=1 for new POW algorithm.
  • the libs should be different in links.txt: -lcryptonote_core -lcryptonote_basic -lcncrypto -lringct -lcrypto -lmultisig -lepee -lcommon -ldevice -lringct_basic -leasylogging and -lunbound at the end
  • -ldl in cnutil.go (dsym was missing for me...)
    Was tested OK with AEON testnet after block 44000 and with xmrig 2.5

@korjavin
Copy link

Hi guys, any plans on this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants